Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSUB-1089: Enable ci jobs for crunch #16

Merged
merged 14 commits into from
Apr 5, 2024

Conversation

atodorov
Copy link
Collaborator

@atodorov atodorov commented Apr 1, 2024

No description provided.

because previous commits didn't leave this file alone but rather made
lots of changes into it and then subsequently removed it.
will add subsequent changes here to make tests trigger on each PR
however this is a good starting point
@atodorov atodorov force-pushed the testing/CSUB-1089-enable-ci-jobs-for-crunch branch 2 times, most recently from 0a67440 to d3141a5 Compare April 4, 2024 09:44
- cargo check
- cargo clippy
- cargo fmt
- cargo test
@atodorov atodorov force-pushed the testing/CSUB-1089-enable-ci-jobs-for-crunch branch from d3141a5 to d2c8cc5 Compare April 4, 2024 09:46
for the upstream code just prefix them with underscore to minimize
differences, for creditcoin/ just remove them
because fmt and clippy appear to be slightly different from version to
version and we'd like to have consistency regardless of how or where
this is built
@atodorov atodorov force-pushed the testing/CSUB-1089-enable-ci-jobs-for-crunch branch 2 times, most recently from cd1889e to e82d52e Compare April 4, 2024 12:00
@atodorov atodorov force-pushed the testing/CSUB-1089-enable-ci-jobs-for-crunch branch from e82d52e to 9ba640d Compare April 4, 2024 12:02
@atodorov atodorov marked this pull request as ready for review April 4, 2024 12:18
@atodorov atodorov merged commit 74973b8 into dev Apr 5, 2024
6 checks passed
@atodorov atodorov deleted the testing/CSUB-1089-enable-ci-jobs-for-crunch branch April 5, 2024 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant